[No QA] Docs updates for E/A#76741 Update message editing steps for mobile composer#90568
[No QA] Docs updates for E/A#76741 Update message editing steps for mobile composer#90568MelvinBot wants to merge 2 commits into
Conversation
Co-authored-by: Alexey Kureev <Kureev@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
HelpDot Documentation ReviewOverall AssessmentThis PR updates the "Edit or Delete Messages" section in Scores Summary
Key Findings
Recommendations
Files Reviewed
Review based on governance files: HELPSITE_NAMING_CONVENTIONS.md, HELP_AUTHORING_GUIDELINES.md, and TEMPLATE.md. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 79d533184d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
HelpDot Documentation ReviewOverall AssessmentThis PR improves the "Edit or Delete Messages" section by splitting a single generic set of steps into platform-specific instructions for desktop and mobile. The change is well-motivated: the interaction patterns genuinely differ between platforms (right-click/hover vs. long-press, inline editing vs. composer-based editing). The new content is clear, accurate, and follows governance conventions with only minor issues. Scores Summary
Key Findings
Recommendations
Files Reviewed
|
|
A preview of your ExpensifyHelp changes have been deployed to https://8f12c9e2.helpdot.pages.dev ⚡️ Updated articles: |
| # Edit or Delete Messages | ||
|
|
||
| **To edit or delete a message:** | ||
| ## How to edit or delete a message on web |
There was a problem hiding this comment.
Heading style: This task-based heading is well-structured (starts with "How to"), but consider including the feature name for better AI retrieval and searchability per HELP_AUTHORING_GUIDELINES.md Section 6 ("Include at least one full 'How to...' heading using feature name").
Suggestion: ## How to edit or delete a chat message on web
This minor addition of "chat" ties the heading to the article's feature name and improves semantic search matching.
HelpDot Documentation ReviewOverall AssessmentThis PR makes targeted improvements to the "Edit or Delete Messages" section of the Chat Features and Moderation Tools article. The changes (1) correct "Desktop" to "Web" for platform labeling consistency, (2) split the edit/delete instructions into separate web and mobile sections with proper Scores Summary
Key Findings
Recommendations
Files Reviewed
Review based on governance files: HELPSITE_NAMING_CONVENTIONS.md, HELP_AUTHORING_GUIDELINES.md, and TEMPLATE.md. |
| 3. Select **Edit comment** or **Delete comment**. | ||
| 4. If editing, update the text inline and press **Enter** to save. | ||
|
|
||
| ## How to edit or delete a message on mobile |
There was a problem hiding this comment.
Heading style: Same suggestion as the web counterpart -- consider including the feature name for searchability.
Suggestion: ## How to edit or delete a chat message on mobile
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? cc @lschurr |
HelpDot Documentation ReviewOverall AssessmentThis PR improves the "Chat Features and Moderation Tools" article by (1) correcting a platform label from "Desktop" to "Web" and (2) splitting the edit/delete message instructions into separate web and mobile sections with platform-specific detail. Both changes align with governance requirements for cross-platform clarity and accurate UI terminology. Scores Summary
Key Findings
Recommendations
Files Reviewed
|
| # Edit or Delete Messages | ||
|
|
||
| **To edit or delete a message:** | ||
| ## How to edit or delete a message on web |
There was a problem hiding this comment.
Naming and style: "web" should be capitalized to "Web" for consistency with the rest of the article (e.g., line 19 uses "On Web:") and standard platform naming.
| ## How to edit or delete a message on web | |
| ## How to edit or delete a message on Web |
| 3. Select **Edit comment** or **Delete comment**. | ||
| 4. If editing, update the text inline and press **Enter** to save. | ||
|
|
||
| ## How to edit or delete a message on mobile |
There was a problem hiding this comment.
Naming and style: "mobile" should be capitalized to "Mobile" for consistency with the rest of the article (e.g., line 26 uses "On Mobile:") and standard platform naming.
| ## How to edit or delete a message on mobile | |
| ## How to edit or delete a message on Mobile |
Summary
Chat-Features-and-Moderation-Tools.mdto reflect the new editing behavior introduced in New editing mechanism for small screens #76741Test plan